Skip to content

Code cleanup#33

Merged
mingxwa merged 3 commits intongcpp:mainfrom
mingxwa:user/mingxwa/refactor
May 9, 2026
Merged

Code cleanup#33
mingxwa merged 3 commits intongcpp:mainfrom
mingxwa:user/mingxwa/refactor

Conversation

@mingxwa
Copy link
Copy Markdown
Collaborator

@mingxwa mingxwa commented May 3, 2026

Changes

  • Updated the wording for "pointer-like type eligible for proxy" in named requirements ProFacade: Removed clause "P is not a specialization of proxy". This does not align with the standard wording style.
  • In the value constructors / assignments of proxy, specified that they won't participate overload resolution if std::decay_t<P> is a specialization of proxy.
  • Split detail::ptr_traits into concept pointer_like and concept specialization_of.
  • Added missing requires clause in swap().
  • Removed redundant <F> from invoke().
  • Added a unit test case for proxiable eligibility.

Comment thread include/proxy/v4/proxy.h Outdated
Comment thread include/proxy/v4/proxy.h Outdated
Comment thread include/proxy/v4/proxy.h
@mingxwa mingxwa requested a review from tian-lt-personal May 8, 2026 14:45
@mingxwa mingxwa merged commit 7400868 into ngcpp:main May 9, 2026
16 checks passed
@mingxwa mingxwa deleted the user/mingxwa/refactor branch May 9, 2026 05:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants